Skip to content

Review and refactor codebase with documentation updates#20

Merged
at-ok merged 1 commit into
mainfrom
claude/codebase-review-refactor-014RfXjs8uCCm3sMAL4vijND
Nov 20, 2025
Merged

Review and refactor codebase with documentation updates#20
at-ok merged 1 commit into
mainfrom
claude/codebase-review-refactor-014RfXjs8uCCm3sMAL4vijND

Conversation

@at-ok

@at-ok at-ok commented Nov 20, 2025

Copy link
Copy Markdown
Owner

主な変更点

1. コード重複の削減

  • 距離計算関数を lib/geometry.ts に統一(5ファイルから約200行削減)
  • ユーザーマッピング関数を lib/user-mapper.ts に統一(3ファイルから約90行削減)

2. 型安全性の向上

  • hooks/useAuth.tsx で as any を完全除去
  • user-mapper を活用した型安全なデータ変換

3. 定数の一元化

  • constants/ui-labels.ts を新規作成(ゲーム状態、ミッション、ロールラベル)
  • constants/game-config.ts を新規作成(ゲーム設定定数)

4. パフォーマンス最適化

  • Map コンポーネントで useCallback によるメモ化
  • gamemaster ページで重複 find() 呼び出しを削減

5. エラーハンドリング強化

  • useLocation でエラー状態を適切に設定
  • API ルートでエラーログを改善

6. ドキュメント更新

  • README.md のディレクトリ構成を更新
  • docs/REFACTORING.md を新規作成(詳細な改善記録)

新規ファイル

  • lib/geometry.ts
  • lib/user-mapper.ts
  • constants/ui-labels.ts
  • constants/game-config.ts
  • docs/REFACTORING.md

効果

  • コード重複: 約300行削減
  • 型安全性: as any 使用箇所 2 → 0
  • 保守性とコード品質の大幅な向上

## 主な変更点

### 1. コード重複の削減
- 距離計算関数を lib/geometry.ts に統一(5ファイルから約200行削減)
- ユーザーマッピング関数を lib/user-mapper.ts に統一(3ファイルから約90行削減)

### 2. 型安全性の向上
- hooks/useAuth.tsx で as any を完全除去
- user-mapper を活用した型安全なデータ変換

### 3. 定数の一元化
- constants/ui-labels.ts を新規作成(ゲーム状態、ミッション、ロールラベル)
- constants/game-config.ts を新規作成(ゲーム設定定数)

### 4. パフォーマンス最適化
- Map コンポーネントで useCallback によるメモ化
- gamemaster ページで重複 find() 呼び出しを削減

### 5. エラーハンドリング強化
- useLocation でエラー状態を適切に設定
- API ルートでエラーログを改善

### 6. ドキュメント更新
- README.md のディレクトリ構成を更新
- docs/REFACTORING.md を新規作成(詳細な改善記録)

## 新規ファイル
- lib/geometry.ts
- lib/user-mapper.ts
- constants/ui-labels.ts
- constants/game-config.ts
- docs/REFACTORING.md

## 効果
- コード重複: 約300行削減
- 型安全性: as any 使用箇所 2 → 0
- 保守性とコード品質の大幅な向上
@at-ok at-ok merged commit 0db42bd into main Nov 20, 2025
1 of 4 checks passed
@at-ok at-ok deleted the claude/codebase-review-refactor-014RfXjs8uCCm3sMAL4vijND branch November 20, 2025 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants